Skip to content

Show watchpoint at tip in quickstart tutorial#752

Open
k4rtikx wants to merge 1 commit intoprecice:developfrom
k4rtikx:quickstart-watchpoint-tip
Open

Show watchpoint at tip in quickstart tutorial#752
k4rtikx wants to merge 1 commit intoprecice:developfrom
k4rtikx:quickstart-watchpoint-tip

Conversation

@k4rtikx
Copy link

@k4rtikx k4rtikx commented Mar 9, 2026

The solid solver already records a watchpoint at the flap tip (configured in precice-config.xml), and a plotting script plotDisplacement.sh already exists, but the README does not mention either.

This PR adds a paragraph to the "Visualizing the results" section explaining how users can plot the flap tip displacement using gnuplot.

Closes #671

@MakisH MakisH added the GSoC Contributed in the context of the Google Summer of Code label Mar 13, 2026
@MakisH
Copy link
Member

MakisH commented Mar 20, 2026

Hi! While the respective change seems to be in cdb9eee, it looks like the source branch got a bit messed up. If you clean it up, I would be happy to merge it.

@k4rtikx
Copy link
Author

k4rtikx commented Mar 21, 2026

Hi @MakisH , thanks for the feedback!

I realized the branch included unrelated commits and changes. I have cleaned it up and isolated the relevant change in a new branch here:
https://github.com/k4rtikx/tutorials/tree/watchpoint-tip-clean

Closing this PR to keep things clean.

@k4rtikx k4rtikx closed this Mar 21, 2026
@MakisH
Copy link
Member

MakisH commented Mar 21, 2026

FYI, an alternative way would have been to git rebase -i develop, drop the unrelated commits, and force-push to the same branch.

Happy to look at it again after the GSoC spike is through.

@k4rtikx
Copy link
Author

k4rtikx commented Mar 21, 2026

Thanks for the suggestion @MakisH
Understood using git rebase -i develop would’ve been the better approach here. I’ll follow that going forward.

Will contribute again once it’s open.

@MakisH MakisH reopened this Mar 21, 2026
@MakisH
Copy link
Member

MakisH commented Mar 21, 2026

@k4rtikx I reopened it, in case you want to try here. If not, feel free to close it again, or I will close it next week.

@k4rtikx k4rtikx force-pushed the quickstart-watchpoint-tip branch from 18ee08d to ae92d8a Compare March 21, 2026 21:00
@k4rtikx
Copy link
Author

k4rtikx commented Mar 21, 2026

@MakisH thanks for reopening this and for the suggestion.

I’ve cleaned up the branch, removed the extra commits, and force-pushed the changes. It should now just contain the relevant update.

Happy to fix anything else if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Contributed in the context of the Google Summer of Code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Quickstart: Show watchpoint at tip

2 participants